-
Notifications
You must be signed in to change notification settings - Fork 6
Added a github workflow for running examples as integration tests #140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Waiting for https://github.com/paritytech/polkadot-bulletin-chain/actions/runs/20095449181 to finish... |
| push: | ||
| branches: [main, ci_cd] | ||
| pull_request: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not a GH CI/CD export, I was just looking for a way to run the workflow from my branch. Now that it's working, I can simply remove my (ci_cd) branch from here and merge, right? @bkontur
Co-authored-by: Branislav Kontur <[email protected]>
Co-authored-by: Branislav Kontur <[email protected]>
examples/justfile
Outdated
|
|
||
| # Run smoldot example with Docker IPFS | ||
| run-authorize-and-store-smoldot-docker: build npm-install | ||
| run-authorize-and-store-smoldot: build npm-install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is duplicate now? We have the same on the line 65?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, I refactored justfile a bit, PTAL
Co-authored-by: Branislav Kontur <[email protected]>
* added Docker flow to justfile * Added a github workflow for running examples as integration tests (#140) * added CI for docker workflow * simplified workflow, reused just commands * doubled timeout * use ubuntu-latest directly for Docker suppor * added docker check * simplify docker verification * trigger workflow on ci_cd branch pushes * add protobuf-compiler dependency * add libclang dependency * add wasm32 target for runtime compilation * add rust-src component for wasm compilation * use host networking for IPFS Docker on Linux/CI * fix Docker networking and add zombienet debugging * fix: use absolute paths for zombienet and bulletin binary * renamed * removed docker verification step - not needed anymore * Update .github/workflows/integration-test.yml Co-authored-by: Branislav Kontur <[email protected]> * Update .github/workflows/integration-test.yml Co-authored-by: Branislav Kontur <[email protected]> * added another test as step * updated on push branches * removed branhces from on pull_request * added ci_cd back, modified network_args (added ports) * removed flag in wget * added docker setup * added missing dependency * commented out first test - debugging second * fixed commit * reverted import, increased node version to 22 * uncommented first test * reverted branches * Update .github/workflows/integration-test.yml Co-authored-by: Branislav Kontur <[email protected]> * removed local set-up * Update examples/justfile * parametrized run-authorize-and-store * removed duplicate * Update examples/justfile Co-authored-by: Branislav Kontur <[email protected]> * Removed older TODO * Extract node version * Actually use CI_IMAGE and align with check.yml --------- Co-authored-by: Branislav Kontur <[email protected]> --------- Co-authored-by: Branislav Kontur <[email protected]>
No description provided.